Skip to content

Conversation

@JWT007
Copy link
Contributor

@JWT007 JWT007 commented Apr 8, 2025

Addresses an issue where non-SOAP HTTP error responses (4xx-5xx)
with a "text/html" content type were not being properly handled,
resulting in uninformative error messages.

Now, the response body from these errors is extracted and included
in the AxisFault detail, providing more context for debugging.
This ensures that users receive more meaningful error information
when such errors occur.

Addresses an issue where non-SOAP HTTP error responses (4xx-5xx)
with a "text/html" content type were not being properly handled,
resulting in uninformative error messages.

Now, the response body from these errors is extracted and included
in the AxisFault detail, providing more context for debugging.
This ensures that users receive more meaningful error information
when such errors occur.
Copy link
Contributor

@MCMicS MCMicS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only would remove the blank lines after return statements

@robertlazarski robertlazarski merged commit c54cc29 into apache:master Apr 14, 2025
4 checks passed
@robertlazarski
Copy link
Contributor

robertlazarski commented Apr 14, 2025

@JWT007 thanks for the contribution - this looks great and is the community solution for this use case that I was hoping for.

@MCMicS thanks for the review - I will remove the extra lines around the return statements in a following commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants